Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[fix](mtmv) mtmv getPlanBySql should not reuse ctx's StatementContext #39690

Merged
merged 1 commit into from
Aug 21, 2024
Merged

[fix](mtmv) mtmv getPlanBySql should not reuse ctx's StatementContext #39690

merged 1 commit into from
Aug 21, 2024

Conversation

morrySnow
Copy link
Contributor

No description provided.

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@morrySnow
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

TPC-H: Total hot run time: 38751 ms 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 3546047aea66aa7042213fcae69744be44c338b7, data reload: false

------ Round 1 ----------------------------------
q1	17867	4415	4335	4335
q2	2065	208	210	208
q3	10450	1228	1139	1139
q4	10718	751	716	716
q5	8137	2896	2854	2854
q6	267	159	164	159
q7	1005	664	648	648
q8	9792	2171	2127	2127
q9	7496	6680	6676	6676
q10	7653	2270	2241	2241
q11	472	269	275	269
q12	434	255	254	254
q13	18544	3010	3023	3010
q14	302	263	253	253
q15	565	536	554	536
q16	556	425	417	417
q17	1015	771	769	769
q18	7498	6862	6751	6751
q19	6922	1041	1147	1041
q20	738	367	361	361
q21	4470	3010	2971	2971
q22	1171	1027	1016	1016
Total cold run time: 118137 ms
Total hot run time: 38751 ms

----- Round 2, with runtime_filter_mode=off -----
q1	4464	4298	4302	4298
q2	404	305	292	292
q3	2906	2635	2664	2635
q4	1935	1582	1642	1582
q5	5386	5405	5359	5359
q6	243	147	148	147
q7	2098	1687	1704	1687
q8	3208	3354	3328	3328
q9	8480	8383	8383	8383
q10	3432	3162	3152	3152
q11	615	535	513	513
q12	816	643	653	643
q13	16762	3023	2992	2992
q14	318	286	301	286
q15	571	523	522	522
q16	503	444	446	444
q17	1789	1513	1504	1504
q18	7671	7609	7411	7411
q19	1768	1607	1797	1607
q20	2052	1861	1816	1816
q21	9723	5097	4969	4969
q22	1127	1050	1062	1050
Total cold run time: 76271 ms
Total hot run time: 54620 ms

@doris-robot
Copy link

TPC-DS: Total hot run time: 190944 ms 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 3546047aea66aa7042213fcae69744be44c338b7, data reload: false

query1	963	396	389	389
query2	6729	1982	1945	1945
query3	6673	232	253	232
query4	33940	23259	23184	23184
query5	4378	719	701	701
query6	317	219	213	213
query7	4625	335	318	318
query8	479	436	436	436
query9	8623	2532	2494	2494
query10	502	359	331	331
query11	16756	15067	15142	15067
query12	187	139	135	135
query13	1707	463	450	450
query14	9962	7199	7062	7062
query15	339	194	194	194
query16	8114	519	487	487
query17	1828	593	586	586
query18	2064	356	338	338
query19	352	177	169	169
query20	141	138	139	138
query21	251	147	140	140
query22	4249	4036	3992	3992
query23	34275	33281	33290	33281
query24	10971	2872	2947	2872
query25	637	416	416	416
query26	1159	187	180	180
query27	2457	305	301	301
query28	7139	2150	2097	2097
query29	836	470	444	444
query30	341	190	187	187
query31	1036	812	802	802
query32	115	83	83	83
query33	815	355	368	355
query34	899	519	502	502
query35	892	768	770	768
query36	1136	960	909	909
query37	178	102	100	100
query38	4071	3874	3873	3873
query39	1535	1460	1452	1452
query40	240	161	159	159
query41	140	138	138	138
query42	134	117	121	117
query43	550	504	495	495
query44	1233	778	808	778
query45	227	198	198	198
query46	1134	773	771	771
query47	1884	1825	1816	1816
query48	418	337	338	337
query49	1195	581	582	581
query50	860	475	462	462
query51	7345	7173	7156	7156
query52	118	108	115	108
query53	301	229	226	226
query54	1033	508	496	496
query55	88	88	86	86
query56	346	305	309	305
query57	1189	1154	1151	1151
query58	329	319	322	319
query59	3001	2861	2818	2818
query60	360	326	347	326
query61	193	148	147	147
query62	863	728	709	709
query63	266	223	228	223
query64	5333	2342	1828	1828
query65	3207	3164	3157	3157
query66	1144	675	677	675
query67	15329	15166	14827	14827
query68	5803	586	598	586
query69	734	428	350	350
query70	1226	1166	1136	1136
query71	549	312	353	312
query72	7747	2297	2058	2058
query73	852	358	361	358
query74	9230	8947	8803	8803
query75	4832	2786	2737	2737
query76	4267	1075	1092	1075
query77	883	445	425	425
query78	9770	9214	9031	9031
query79	3991	552	553	552
query80	2532	619	608	608
query81	625	315	262	262
query82	804	161	164	161
query83	348	217	218	217
query84	300	101	127	101
query85	1532	361	356	356
query86	440	316	329	316
query87	4456	4196	4187	4187
query88	4660	2513	2498	2498
query89	463	337	322	322
query90	2062	235	234	234
query91	155	134	130	130
query92	87	79	76	76
query93	4538	579	556	556
query94	991	315	321	315
query95	387	296	301	296
query96	620	287	285	285
query97	3252	3030	3079	3030
query98	249	231	226	226
query99	1607	1313	1326	1313
Total cold run time: 310123 ms
Total hot run time: 190944 ms

@doris-robot
Copy link

ClickBench: Total hot run time: 30.86 s 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit 3546047aea66aa7042213fcae69744be44c338b7, data reload: false

query1	0.04	0.05	0.04
query2	0.09	0.04	0.04
query3	0.22	0.05	0.06
query4	1.68	0.10	0.09
query5	0.53	0.50	0.49
query6	1.13	0.75	0.73
query7	0.02	0.01	0.01
query8	0.05	0.05	0.05
query9	0.54	0.49	0.49
query10	0.55	0.55	0.55
query11	0.16	0.13	0.13
query12	0.16	0.13	0.14
query13	0.62	0.60	0.59
query14	0.77	0.78	0.77
query15	0.84	0.82	0.83
query16	0.38	0.38	0.37
query17	0.98	1.00	1.00
query18	0.21	0.20	0.21
query19	1.80	1.66	1.70
query20	0.01	0.01	0.02
query21	15.40	0.67	0.66
query22	5.50	6.32	1.62
query23	18.31	1.43	1.24
query24	2.13	0.23	0.22
query25	0.17	0.08	0.08
query26	0.28	0.19	0.19
query27	0.08	0.08	0.08
query28	13.22	1.03	1.02
query29	12.61	3.39	3.35
query30	0.43	0.25	0.25
query31	2.80	0.41	0.42
query32	3.24	0.49	0.51
query33	2.92	3.00	2.96
query34	17.08	4.37	4.39
query35	4.44	4.45	4.49
query36	0.67	0.51	0.49
query37	0.21	0.18	0.18
query38	0.18	0.18	0.18
query39	0.08	0.06	0.06
query40	0.19	0.16	0.15
query41	0.11	0.07	0.07
query42	0.08	0.07	0.06
query43	0.07	0.07	0.06
Total cold run time: 110.98 s
Total hot run time: 30.86 s

@github-actions GitHub Actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Aug 21, 2024
@github-actions GitHub Actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@morrySnow morrySnow merged commit bddf728 into apache:master Aug 21, 2024
29 of 31 checks passed
@morrySnow morrySnow deleted the fix_mtmv_task branch August 21, 2024 07:05
morrySnow added a commit that referenced this pull request Aug 21, 2024
@morrySnow
[fix](mtmv) mtmv getPlanBySql should not reuse ctx's StatementContext (…
08ba881 
…#39690)

pick from master #39690
@morrySnow morrySnow mentioned this pull request Aug 21, 2024
Merged
morrySnow added a commit that referenced this pull request Aug 21, 2024
@morrySnow
[fix](mtmv) mtmv getPlanBySql should not reuse ctx's StatementContext (…
2890a98 
…#39690) (#39719)

pick from master #39690
@yiguolei yiguolei mentioned this pull request Sep 5, 2024
Open
3 tasks
dataroaring pushed a commit that referenced this pull request Sep 26, 2024
@morrySnow @dataroaring
[fix](mtmv) mtmv getPlanBySql should not reuse ctx's StatementContext (…
7070428 
…#39690)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zddr zddr zddr approved these changes

@starocean999 starocean999 starocean999 approved these changes

@seawinde seawinde seawinde approved these changes

Assignees
No one assigned
Labels
approved Indicates a PR has been approved by one committer. dev/2.1.6-merged dev/3.0.2-merged p0_b reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@morrySnow @doris-robot @zddr @starocean999 @seawinde @dataroaring